Skip to content

Reword the recommendation to use Suspenders#760

Merged
heyvaleria merged 4 commits into
mainfrom
edit_recommendation_suspenders
May 29, 2025
Merged

Reword the recommendation to use Suspenders#760
heyvaleria merged 4 commits into
mainfrom
edit_recommendation_suspenders

Conversation

@heyvaleria
Copy link
Copy Markdown
Contributor

After this Slack conversation, we realised that Suspenders is not currently recommended after all.
https://thoughtbot.slack.com/archives/C05DN55CPL3/p1747661858395259

Copy link
Copy Markdown
Contributor

@stevepolitodesign stevepolitodesign left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd love to get more feedback on this, as I think the only controversial take is we're now suggesting we prefer Minitest over RSpec for new applications. There's an open discussion about this.

Comment thread rails/how-to/start_a_new_rails_app.md Outdated
Comment thread rails/how-to/start_a_new_rails_app.md Outdated
@heyvaleria
Copy link
Copy Markdown
Contributor Author

Thanks for you input, @stevepolitodesign
That's why I wished you opened this PR 😉

I did want to get this going though, because I do not want to find myself or anyone else starting a new app for a client and running into all those issues by using suspenders, and getting a bunch or extra stuff we likely don't need in a fresh rails app.
It is extra time and extra struggle, and it doesn't look good with the client either.

@stevepolitodesign
Copy link
Copy Markdown
Contributor

I'll also add that we have not used Suspenders for several new Team Rocker projects, so we're already not following the guidance.

@heyvaleria
Copy link
Copy Markdown
Contributor Author

cc @samithoughtbot @slickepinne ☝️

Copy link
Copy Markdown
Contributor

@samithoughtbot samithoughtbot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@heyvaleria thank you for taking the time to clarify this! This will help others going forward!

Comment thread rails/how-to/start_a_new_rails_app.md
heyvaleria and others added 4 commits May 26, 2025 09:53
Co-authored-by: Steve Polito <stevepolitodesign@users.noreply.github.com>
Co-authored-by: Steve Polito <stevepolitodesign@users.noreply.github.com>
@heyvaleria heyvaleria force-pushed the edit_recommendation_suspenders branch from fe24604 to 9daa37b Compare May 26, 2025 07:53
@heyvaleria heyvaleria merged commit 0dc501c into main May 29, 2025
2 checks passed
@heyvaleria heyvaleria deleted the edit_recommendation_suspenders branch May 29, 2025 07:53
stevepolitodesign added a commit that referenced this pull request Dec 19, 2025
Follow-up top #760.

The [latest release][release] is production ready!

This reverts commit 0dc501c, and
adjusts the recommend command.

[release]: https://github.com/thoughtbot/suspenders/releases/tag/v20251219.0
stevepolitodesign added a commit that referenced this pull request Jan 5, 2026
Follow-up top #760.

The [latest release][release] is production ready!

This reverts commit 0dc501c, and
adjusts the recommend command.

[release]: https://github.com/thoughtbot/suspenders/releases/tag/v20251219.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants